Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PROGRESS: main.py splitting #27

Merged
merged 5 commits into from
Jan 19, 2024
Merged

PROGRESS: main.py splitting #27

merged 5 commits into from
Jan 19, 2024

Conversation

OcelotWalrus
Copy link
Member

@OcelotWalrus OcelotWalrus commented Jan 19, 2024

Feature

This PR enhance the game source code by splitting the main.py script into other scrips depending on the handling type.

Summary

This PR adds:

    • data_handling.py script, for handling data loading on game loop start
    • npc_handling.py script, for handling npc actions
    • zone_handling.py script, for handling hostels, stables etc. action
    • text_handling.py script, for handling basic functions used over many classes
    • weapon_upgrade_handling.py script, for handling basic functions used to upgrade work
    • Fix small long term goal 'Make buying multiple metals at once possible in forges' from this wiki page

Performance Impact

N/A

@OcelotWalrus OcelotWalrus added enhancement New feature or request help wanted Extra attention is needed to be fixed Issues that needs the bug to be fixed ASAP labels Jan 19, 2024
@OcelotWalrus OcelotWalrus self-assigned this Jan 19, 2024
@OcelotWalrus OcelotWalrus marked this pull request as ready for review January 19, 2024 17:12
@OcelotWalrus OcelotWalrus merged commit 2d55885 into master Jan 19, 2024
7 of 8 checks passed
@OcelotWalrus OcelotWalrus deleted the main-splitting branch January 19, 2024 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed to be fixed Issues that needs the bug to be fixed ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant